Skip to content

fix(aztec-up): add sensible defaults to installer y/n prompts#21824

Merged
nchamo merged 1 commit intomerge-train/fairiesfrom
fix/installer-prompt-defaults
Mar 20, 2026
Merged

fix(aztec-up): add sensible defaults to installer y/n prompts#21824
nchamo merged 1 commit intomerge-train/fairiesfrom
fix/installer-prompt-defaults

Conversation

@nchamo
Copy link
Contributor

@nchamo nchamo commented Mar 19, 2026

Problem

The Aztec installer prompts show (y/n) with no indicated default. Pressing Enter at any prompt aborts silently, which is confusing -- especially the fresh-install prompt where the user clearly intends to proceed.

Fix

Give each prompt a sensible default and surface it in the prompt text using the standard Y/n (default-yes) and y/N (default-no) convention.

Prompt File Default Behavior on Enter
Fresh install confirmation aztec-install Yes (Y/n) Proceeds with install
Remove old install aztec-install No (y/N) Aborts (destructive action)
Prune old versions aztec-up No (y/N) Aborts (destructive action)

Fixes F-445

@nchamo nchamo requested a review from charlielye as a code owner March 19, 2026 22:22
@nchamo nchamo self-assigned this Mar 19, 2026
@nchamo nchamo requested a review from nventuro March 19, 2026 22:35
@nchamo nchamo merged commit c268385 into merge-train/fairies Mar 20, 2026
25 checks passed
@nchamo nchamo deleted the fix/installer-prompt-defaults branch March 20, 2026 14:19
@AztecBot
Copy link
Collaborator

❌ Failed to cherry-pick to v4-next due to conflicts. (🤖) View backport run.

AztecBot pushed a commit that referenced this pull request Mar 20, 2026
AztecBot pushed a commit that referenced this pull request Mar 20, 2026
nchamo pushed a commit that referenced this pull request Mar 20, 2026
github-merge-queue bot pushed a commit that referenced this pull request Mar 20, 2026
BEGIN_COMMIT_OVERRIDE
refactor: revert remove assert_bounded_vec_trimmed (#21758)
feat!: no_from (#21716)
fix(bot): use NO_FROM instead of AztecAddress.ZERO in deploy simulate
(#21402)
feat!: make isContractInitialized a tri-state enum (#21754)
feat: sync poseidon in the browser (#21833)
fix(aztec-up): add sensible defaults to installer y/n prompts (#21824)
feat: sync poseidon browser (#21851)
END_COMMIT_OVERRIDE
AztecBot added a commit that referenced this pull request Mar 21, 2026
BEGIN_COMMIT_OVERRIDE
chore: backport #21754 (feat!: make isContractInitialized a tri-state
enum) to v4-next (#21792)
fix(stdlib): zero-pad bufferFromFields when declared length exceeds
payload (#21802)
test(protocol-contracts): verify max-size bytecode fits in contract
class log (#21818)
chore: port P2P mesh topic deflake fix to v4-next (#21825)
fix(archiver): throw on duplicate contract class or instance additions
(#21799)
feat: sync poseidon in the browser (#21833)
chore: backport #21824 (fix(aztec-up): add sensible defaults to
installer y/n prompts) to v4-next (#21844)
fix(sequencer): backport wall-clock time for slot estimation to v4-next
(#21769) (#21847)
chore: backport PR #21788 (feat(p2p): add tx validation for contract
class id verification) to v4-next (#21852)
feat: sync poseidon browser (#21851)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants